CONSOLE-5281: Migrate Cypress E2E tests to Playwright (batch 2) by rhamilto · Pull Request #16454 · openshift/console

rhamilto · 2026-05-15T15:54:47Z

Includes changes from #16444

Summary

Migrates CRD extensions, events, favorites, i18n pseudolocalization, alertmanager, and OLM create-namespace Cypress tests to Playwright
Removes orphaned Cypress alertmanager test files and the alertmanager.ts view file
Adds data-test attributes alongside existing data-test-id attributes on utilization cards, text filter, and external links to enable getByTestId usage
Adds createPod and deletePod methods to KubernetesClient
Adds AlertmanagerPage page object and navigation helpers
Uses condition-based waits throughout (no waitForTimeout or networkidle anti-patterns)
Accepts K8sModel | undefined in useCRDAdditionalPrinterColumns hook signature

Tests migrated (19 Cypress → 19 Playwright)

Cypress source	Playwright output
`integration-tests/tests/crd-extensions/console-cli-download.cy.ts`	`e2e/tests/console/crd-extensions/console-cli-download.spec.ts`
`integration-tests/tests/crd-extensions/console-external-log-link.cy.ts`	`e2e/tests/console/crd-extensions/console-external-log-link.spec.ts`
`integration-tests/tests/crd-extensions/console-link.cy.ts`	`e2e/tests/console/crd-extensions/console-link.spec.ts`
`integration-tests/tests/crd-extensions/console-notification.cy.ts`	`e2e/tests/console/crd-extensions/console-notification.spec.ts`
`integration-tests/tests/crd-extensions/console-yaml-sample.cy.ts`	`e2e/tests/console/crd-extensions/console-yaml-sample.spec.ts`
`integration-tests/tests/events/events.cy.ts`	`e2e/tests/console/events/events.spec.ts`
`integration-tests/tests/favorite/favorite-option.cy.ts`	`e2e/tests/console/favorites/favorites.spec.ts`
`integration-tests/tests/i18n/pseudolocalization.cy.ts`	`e2e/tests/console/i18n/pseudolocalization.spec.ts`
`olm/integration-tests/tests/create-namespace.cy.ts`	`e2e/tests/olm/create-namespace.spec.ts`
`integration-tests/tests/cluster-settings/alertmanager/alertmanager.cy.ts`	`e2e/tests/console/cluster-settings/alertmanager/alertmanager.spec.ts`
`integration-tests/tests/cluster-settings/alertmanager/receivers/email.cy.ts`	`e2e/tests/console/cluster-settings/alertmanager/receivers/email.spec.ts`
`integration-tests/tests/cluster-settings/alertmanager/receivers/pagerduty.cy.ts`	`e2e/tests/console/cluster-settings/alertmanager/receivers/pagerduty.spec.ts`
`integration-tests/tests/cluster-settings/alertmanager/receivers/slack.cy.ts`	`e2e/tests/console/cluster-settings/alertmanager/receivers/slack.spec.ts`
`integration-tests/tests/cluster-settings/alertmanager/receivers/webhook.cy.ts`	`e2e/tests/console/cluster-settings/alertmanager/receivers/webhook.spec.ts`

Cypress cleanup

Deleted integration-tests/views/alertmanager.ts (only consumed by the migrated alertmanager tests)

Source component changes

text-filter.tsx: Added data-test="item-filter" alongside data-test-id
utilization-card.tsx (3 files): Added data-test="utilization-card" alongside data-test-id
useCRDAdditionalPrinterColumns.ts: Fixed react-hooks/exhaustive-deps lint warning; updated model param to accept K8sModel | undefined
ExternalLink.tsx: Added data-test attribute

Screenshots / Screen recordings

Automated test migration — no visual changes

Test plan

All migrated Playwright tests pass locally (CRD extensions, events, favorites, i18n, alertmanager main, OLM)
Alertmanager receiver tests have pre-existing flakes unrelated to migration (identical results with original Cypress cy.wait(10000))
TypeScript type-checking passes
ESLint passes
CI passes
No orphaned resources after test runs

🤖 Generated with Claude Code

openshift-ci-robot · 2026-05-15T15:54:51Z

@rhamilto: This pull request references CONSOLE-5281 which is a valid jira issue.

Warning: The referenced jira issue has an invalid target version for the target branch this PR targets: expected the story to target the "5.0.0" version, but no target version was set.

Details

In response to this:

Summary

Migrates CRD extensions, events, favorites, i18n pseudolocalization, alertmanager, and OLM create-namespace Cypress tests to Playwright

Adds data-test attributes alongside existing data-test-id attributes on utilization cards, text filter, and console select items to enable getByTestId usage

Adds createPod and deletePod methods to KubernetesClient

Adds AlertmanagerPage page object and navigation helpers

Tests migrated (19 Cypress → 19 Playwright)

Cypress source Playwright output

integration-tests/tests/crd-extensions/console-cli-download.cy.ts e2e/tests/console/crd-extensions/console-cli-download.spec.ts

integration-tests/tests/crd-extensions/console-external-log-link.cy.ts e2e/tests/console/crd-extensions/console-external-log-link.spec.ts

integration-tests/tests/crd-extensions/console-link.cy.ts e2e/tests/console/crd-extensions/console-link.spec.ts

integration-tests/tests/crd-extensions/console-notification.cy.ts e2e/tests/console/crd-extensions/console-notification.spec.ts

integration-tests/tests/crd-extensions/console-yaml-sample.cy.ts e2e/tests/console/crd-extensions/console-yaml-sample.spec.ts

integration-tests/tests/events/events.cy.ts e2e/tests/console/events/events.spec.ts

integration-tests/tests/favorite/favorite-option.cy.ts e2e/tests/console/favorites/favorites.spec.ts

integration-tests/tests/i18n/pseudolocalization.cy.ts e2e/tests/console/i18n/pseudolocalization.spec.ts

olm/integration-tests/tests/create-namespace.cy.ts e2e/tests/olm/create-namespace.spec.ts

integration-tests/tests/alertmanager/alertmanager.cy.ts e2e/tests/console/cluster-settings/alertmanager/alertmanager.spec.ts

integration-tests/tests/alertmanager/receivers/email.cy.ts e2e/tests/console/cluster-settings/alertmanager/receivers/email.spec.ts

integration-tests/tests/alertmanager/receivers/pagerduty.cy.ts e2e/tests/console/cluster-settings/alertmanager/receivers/pagerduty.spec.ts

integration-tests/tests/alertmanager/receivers/slack.cy.ts e2e/tests/console/cluster-settings/alertmanager/receivers/slack.spec.ts

integration-tests/tests/alertmanager/receivers/webhook.cy.ts e2e/tests/console/cluster-settings/alertmanager/receivers/webhook.spec.ts

Source component changes

text-filter.tsx: Added data-test="item-filter" alongside data-test-id

console-select.tsx: Changed shared data-test="console-select-item" to unique data-test={console-select-item-${itemKey}}

utilization-card.tsx (3 files): Added data-test="utilization-card" alongside data-test-id

useCRDAdditionalPrinterColumns.ts: Fixed react-hooks/exhaustive-deps lint warning

ExternalLink.tsx: Added data-test attribute

Test plan

All migrated Playwright tests pass locally

CI passes

No orphaned resources after test runs

🤖 Generated with Claude Code

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the openshift-eng/jira-lifecycle-plugin repository.

openshift-ci · 2026-05-15T15:54:54Z

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

rhamilto · 2026-05-19T15:56:43Z

/unhold

rhamilto · 2026-05-19T15:57:11Z

tech debt
/label px-approved
/label docs-approved
/verified by @rhamilto

openshift-ci-robot · 2026-05-19T15:57:23Z

@rhamilto: This PR has been marked as verified by @rhamilto.

Details

In response to this:

tech debt
/label px-approved
/label docs-approved
/verified by @rhamilto

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the openshift-eng/jira-lifecycle-plugin repository.

rhamilto · 2026-05-19T16:27:13Z

/retest

…anager, and OLM Cypress tests to Playwright Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com> Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

rhamilto · 2026-05-20T13:18:39Z

Comments addressed. Thanks, @stefanonardo.

stefanonardo · 2026-05-20T14:04:34Z

/lgtm

openshift-ci · 2026-05-20T14:09:38Z

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: rhamilto, stefanonardo

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details

Needs approval from an approver in each of these files:

~~frontend/OWNERS~~ [rhamilto]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

rhamilto · 2026-05-20T16:34:43Z

/retest

rhamilto · 2026-05-20T20:24:05Z

/retest

rhamilto · 2026-05-21T11:29:46Z

/retest

rhamilto · 2026-05-21T14:56:54Z

/retest

rhamilto · 2026-05-21T17:40:25Z

/retest

rhamilto · 2026-05-26T11:43:35Z

/retest

rhamilto · 2026-05-26T14:31:32Z

/retest

rhamilto · 2026-05-26T16:49:45Z

/retest

rhamilto · 2026-05-26T17:12:38Z

/verified by CI

openshift-ci-robot · 2026-05-26T17:12:51Z

@rhamilto: This PR has been marked as verified by CI.

Details

In response to this:

/verified by CI

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the openshift-eng/jira-lifecycle-plugin repository.

rhamilto · 2026-05-26T19:39:05Z

/retest

openshift-merge-bot · 2026-05-26T22:58:59Z

/retest-required

Remaining retests: 0 against base HEAD 397d1b3 and 2 for PR HEAD cdeec4b in total

openshift-merge-bot · 2026-05-27T01:58:04Z

/retest-required

Remaining retests: 0 against base HEAD b333b29 and 1 for PR HEAD cdeec4b in total

openshift-merge-bot · 2026-05-27T05:00:58Z

/retest-required

Remaining retests: 0 against base HEAD c0ecf9f and 0 for PR HEAD cdeec4b in total

openshift-merge-bot · 2026-05-27T07:59:10Z

/hold

Revision cdeec4b was retested 3 times: holding

rhamilto · 2026-05-27T11:37:35Z

/retest
/unhold

rhamilto · 2026-05-27T14:15:52Z

/retest

openshift-ci · 2026-05-27T16:34:46Z

@rhamilto: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name	Commit	Details	Required	Rerun command
ci/prow/e2e-gcp-console	`cdeec4b`	link	true	`/test e2e-gcp-console`

Full PR test history. Your PR dashboard.

Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

openshift-ci-robot added the jira/valid-reference Indicates that this PR references a valid Jira ticket of any type. label May 15, 2026

openshift-ci Bot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label May 15, 2026

rhamilto force-pushed the CONSOLE-5281 branch 3 times, most recently from 074fbfd to 0b4f078 Compare May 18, 2026 17:31

rhamilto changed the title ~~WIP: CONSOLE-5281: Migrate Cypress E2E tests to Playwright (batch 2)~~ CONSOLE-5281: Migrate Cypress E2E tests to Playwright (batch 2) May 18, 2026

rhamilto force-pushed the CONSOLE-5281 branch from 0b4f078 to 5d398da Compare May 18, 2026 18:10

openshift-ci Bot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label May 19, 2026

rhamilto force-pushed the CONSOLE-5281 branch from 5d398da to c04f9a0 Compare May 19, 2026 12:00

openshift-ci Bot removed the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label May 19, 2026

rhamilto force-pushed the CONSOLE-5281 branch from c04f9a0 to 4711371 Compare May 19, 2026 12:25

rhamilto marked this pull request as ready for review May 19, 2026 13:48

openshift-ci Bot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label May 19, 2026

openshift-ci Bot requested review from spadgett and stefanonardo May 19, 2026 13:49

openshift-ci Bot added px-approved Signifies that Product Support has signed off on this PR docs-approved Signifies that Docs has signed off on this PR labels May 19, 2026

openshift-ci-robot added the verified Signifies that the PR passed pre-merge verification criteria label May 19, 2026

openshift-ci Bot added the tide/merge-method-squash Denotes a PR that should be squashed by tide when it merges. label May 20, 2026

stefanonardo reviewed May 20, 2026

View reviewed changes

Comment thread frontend/e2e/clients/kubernetes-client.ts Outdated

Comment thread frontend/e2e/clients/kubernetes-client.ts Outdated

Comment thread frontend/e2e/pages/alertmanager-page.ts Outdated

CONSOLE-5281: Migrate CRD extensions, events, favorites, i18n, alertm…

cdeec4b

…anager, and OLM Cypress tests to Playwright Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com> Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

rhamilto force-pushed the CONSOLE-5281 branch from 3ef248e to cdeec4b Compare May 20, 2026 13:17

openshift-ci Bot assigned stefanonardo May 20, 2026

openshift-ci Bot added the lgtm Indicates that a PR is ready to be merged. label May 20, 2026

openshift-ci-robot added the verified Signifies that the PR passed pre-merge verification criteria label May 26, 2026

openshift-ci Bot added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label May 27, 2026

openshift-ci Bot removed the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label May 27, 2026

Conversation

rhamilto commented May 15, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary

Tests migrated (19 Cypress → 19 Playwright)

Cypress cleanup

Source component changes

Screenshots / Screen recordings

Test plan

Uh oh!

openshift-ci-robot commented May 15, 2026 • edited by openshift-ci Bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary

Tests migrated (19 Cypress → 19 Playwright)

Source component changes

Test plan

Uh oh!

openshift-ci Bot commented May 15, 2026

Uh oh!

rhamilto commented May 19, 2026

Uh oh!

rhamilto commented May 19, 2026

Uh oh!

openshift-ci-robot commented May 19, 2026

Uh oh!

rhamilto commented May 19, 2026

Uh oh!

Uh oh!

Uh oh!

Uh oh!

rhamilto commented May 20, 2026

Uh oh!

stefanonardo commented May 20, 2026

Uh oh!

openshift-ci Bot commented May 20, 2026

Uh oh!

rhamilto commented May 20, 2026

Uh oh!

rhamilto commented May 20, 2026

Uh oh!

rhamilto commented May 21, 2026

Uh oh!

rhamilto commented May 21, 2026

Uh oh!

rhamilto commented May 21, 2026

Uh oh!

rhamilto commented May 26, 2026

Uh oh!

rhamilto commented May 26, 2026

Uh oh!

rhamilto commented May 26, 2026

Uh oh!

rhamilto commented May 26, 2026

Uh oh!

openshift-ci-robot commented May 26, 2026

Uh oh!

rhamilto commented May 26, 2026

Uh oh!

openshift-merge-bot Bot commented May 26, 2026

Uh oh!

openshift-merge-bot Bot commented May 27, 2026

Uh oh!

openshift-merge-bot Bot commented May 27, 2026

Uh oh!

openshift-merge-bot Bot commented May 27, 2026

Uh oh!

rhamilto commented May 27, 2026

Uh oh!

rhamilto commented May 27, 2026

Uh oh!

openshift-ci Bot commented May 27, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

rhamilto commented May 15, 2026 •

edited

Loading

openshift-ci-robot commented May 15, 2026 •

edited by openshift-ci Bot

Loading